Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #87 #88

Merged
merged 1 commit into from
Sep 5, 2019
Merged

Fixes #87 #88

merged 1 commit into from
Sep 5, 2019

Conversation

fefo-p
Copy link
Contributor

@fefo-p fefo-p commented Aug 31, 2019

Added fallback to username when a non-existant user tries to log in

Added fallback to username when a non-existant user tries to log in
@fefo-p
Copy link
Contributor Author

fefo-p commented Aug 31, 2019

I don't understand what travis is complaining about ...

Failed test result:

  Problem 1
202    - phpunit/phpunit 7.5.x-dev requires php ^7.1 -> your PHP version (8.0.0-dev) does not satisfy that requirement.
203    - phpunit/phpunit 7.5.9 requires php ^7.1 -> your PHP version (8.0.0-dev) does not satisfy that requirement.
204...

While my php version is:

dev-server:myth-auth username$ php -v
PHP 7.3.8 (cli) (built: Aug  6 2019 23:15:07) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.3.8, Copyright (c) 1998-2018 Zend Technologies
    with Zend OPcache v7.3.8, Copyright (c) 1999-2018, by Zend Technologies

@lonnieezell
Copy link
Owner

lonnieezell commented Aug 31, 2019 via email

@fefo-p
Copy link
Contributor Author

fefo-p commented Aug 31, 2019

oh, great!

@fefo-p
Copy link
Contributor Author

fefo-p commented Sep 4, 2019

Any update on this @lonnieezell ?
Should I send a PR with [ciskip] ?

@lonnieezell
Copy link
Owner

Sorry - just haven't had a chance to get back to it. it looks fine, though.

My initial thought was that email should always be the defining characteristic, then I realized the system would allow either one (or anything else, really) to be the dominant fields, so it's all good :)

@lonnieezell
Copy link
Owner

Oh - and please don't use [ci skip] for any code/logic changes. Only for docs/comments, etc.

@lonnieezell lonnieezell merged commit c8cf1fd into lonnieezell:develop Sep 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants